stack: Add counterparts of OVER_UP_DOWN transition

The GtkStackTransitionType enum now has OVER_DOWN_UP, OVER_LEFT_RIGHT,
and OVER_RIGHT_LEFT values to complement OVER_UP_DOWN.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

stack: More readability when adding transitions

This small refactor makes the code more readable when adding more
transition types that have left, right, up, and down variations.

It adds inline boolean functions to tell information about transition
types (avoiding long if clauses) and changes long chains of "else if
(transition_type == ...)" into switch statements. Both are only likely
to get longer as more transition types are added.

https://bugzilla.gnome.org/show_bug.cgi?id=726676

overlay: don't have a custom draw implementation

We can't think with Benjamin of a reason this should be needed, so let's
just remove it.

themingengine: only round outline corners after shrinking

Ensures the correctness of the applied radius.

GtkBuilder: added template parent type check.

GtkFlowBox: fixed NULL dereferencing segfaults

GtkMenu: fixed more NULL dereferencing segfault

GtkGrid: gtk_grid_get_size_for_size() do not dereference a NULL pointer.

GtkBuilder: improved parsing error report for invalid properties and signals.

Added GTK_BUILDER_ERROR_INVALID_PROPERTY and GTK_BUILDER_ERROR_INVALID_SIGNAL
error codes

ObjectInfo: Use a GType instead of a char * for the class name.
PropertyInfo: Use a GParamSpec instead of a char * for the property name.
SignalInfo: Use signal id and detail quark instead of a detailed signal name string.

This not only save us a few malloc in each case but lets us simplify the code
and report unknown properties and signals as a parsing error instead of just
printing a warning.

GtkBuilder: fixed a few minor memory leaks.

Fixed memory leaks in parse_object(), parse_template() and parse_signal() functions.
Parameters value where strduped before the last posible return and not freed.

W32: override selected button style to remove colored background

https://bugzilla.gnome.org/show_bug.cgi?id=729353

W32: Add a theme for .app-notification

This is copied from Adwaita. Looks sufficiently ok.

https://bugzilla.gnome.org/show_bug.cgi?id=729346

listbox: fix a segfault

Don't dereference a NULL pointer.

combobox: fix a segfault

Don't dereference a NULL pointer.

themingengine: use outlines to render focus rings

Instead of the old non-CSS implementation. This allows us to get rid of
the very last piece of custom theming in Adwaita.

css: add properties for outline-radius

Both a shorthand and individual properties. This is not officially part
of the CSS standard, but there's precedence for it at least in Mozilla:
https://developer.mozilla.org/en-US/docs/Web/CSS/-moz-outline-radius

treeview: respect focus-padding when rendering focus

Adwaita had a hack to work this around, but now that we're moving focus
ring rendering into GTK, we can just fix the treeview instead.

treeview: Always call size request funcs with valid out pointers

Not doing so causes crashes since
a158a2aa48c8023f99963642cc2657bff207b82NULLd

revealer: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

popover: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menuitem: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

menu: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

levelbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

eventbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

listbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

frame: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

headerbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

infobar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

grid: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

scrolledwindow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

switch: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

toolbar: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

treeview: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

viewport: Implement hfw properly

Previously, the code would fall back on GtkBin's hfw implementation
which didn't account for padding and borders.

viewport: Use a convenience function

That function looks longer than not using it, but we'll need it in the
next commit.

viewport: Request 2x border-width

It's nice that bugs get pointed out in code comments. Fixing them would
probably have been preferrable though.

widget: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

flowbox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

combobox: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

button: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

alignment: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

arrow: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

spinner: Remove unneeded checks

Size vfuncs always get non-null out variables passed, so no need to
check for NULL.

modules: remove the pixbuf engine directory

This is not used anywhere, and is commented out of the build because it
doesn't compile. Time for some spring cleaning...

Updated POTFILES.skip

GtkSearchBar: fix the template

There was a mismatch between GtkSearchBar being derived from GtkBin
and the template claiming the parent to be GtkBox.

widget-factory: populate the second page more

Add back a menubar example that was lost a while ago, and also
include a searchbar and an infobar.

widget-factory: rearrange page 2 a bit

reftests: Add test for latest commit

Check that the label gets linebroken properly.

window: Subtract shadow from passed in width for height

... and height for width in size requests. Fixes mislayout in
control-center universal access panel.

widgets: Remove (GtkTickCallback) casts

Instead, make the functions conform to the prototype, so that casting
isn't needed.

GtkMenuSectionBox: Don't show separators for nested subsections

...unless they are labeled.

To implement this, keep track of the nesting depth of section boxes,
starting from 0 for the toplevel and the submenu ones, and only
insert unlabeled separators if the depth is at most 1.

GtkMenuSectionBox: Always show separators for labeled sections

...unless they are empty.

Add a comment with rules for separators

Added template test case to builder test.

GtkMenuSectionBox: don't leak

This wasn't fully fleshed out yet.

widget-factory: Add a horizontal-buttons example

Sadly, we don't have good icons for payment methods...

treeview: Fix prelight redraw when adjustment changes

When the adjustment changes (due to e.g. a mouse wheel scroll) we update
the prelight. The part that un-prelights the previous prelight was
broken by the the pixel cache, as it called update_prelight in the
middle of the scrolling operation, where the windows were moved
but the tree_view->priv->dy was not changed to the new value. This
caused the updates to the pixel cache to go to the wrong place.

We fix this by fully doing the scroll before we update_prelight().

https://bugzilla.gnome.org/show_bug.cgi?id=728284

3.13.1

Update output of a11y tests

These are all affected by the addition of the action_box in
GtkDialog or by the switch to popovers in GtkMenuButton.

GtkMenuShell: We want separators in submenus

This was probably an oversight when adding the merge_separators
parameter. Without this change, the testsuite/gtk/gtkmenu test
fails.

GtkMenuShell: Don't use a non-existing property

GtkMenuTrackerItem::visible was removed a few commits ago.
It is not necessary to bind visible anyway, since the menu
tracker will insert and remove items as their visibility
changes.

testmenubutton: Add a toggle for use-popover

gdkwindow: ignore shape on client-side windows when recomputing visible regions

Rendering doesn't do much about clipping drawing operations to the window shape,
although invalidation applies the shape to every window, leaving possibly trails
of "overrendered" content. So ensure the shape portions get invalidated too when
the window is moved/resized.

https://bugzilla.gnome.org/show_bug.cgi?id=729095

popover: Always make room for the tail on all sides on size request

Now that popovers may snap to any side with enough space, make enough
room on every side when requesting size, so that there's no w/h differences
at the time of setting the child allocation.

https://bugzilla.gnome.org/show_bug.cgi?id=729097

GtkRecentChooserMenu: Set current uri before activating an item.

When activating an item using mnemonics GtkRecentChooserMenu does not select a
item before calling the item-activated signal thus
gtk_recent_chooser_get_current_uri() always return the last selected item
instead of the activated one.

Fixes Bug 495105 "Open recent file keyboard shortcuts do not work correctly"

Raleigh: fix a typo

GtkDialog: protect against show_all

The real reason for the problem that the just reverted change
was supposed to address is that testfilechooser uses show_all
to tease out places where composite dialogs don't properly
protect their internals against unintended showing. Well,
lets do that, then.

Revert "GtkFileChooserDialog: Avoid a bottom border in the dialog"

This reverts commit d871105fa8fd20f9be0d07953ee2dd7d90b4c84f.

Updated Slovenian translation

Updated Slovenian translation

Document iconic section support

https://bugzilla.gnome.org/show_bug.cgi?id=727477

Add an example of iconic rendering

testpopover now shows several examples of icon buttons.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkMenuSectionBox: add support for iconic sections

Sections that have the display-hint attribute set to
"horizontal-boxes" will be rendered as a row of icon
buttons, instead of menuitem-like.

GtkMenuTrackerItem: Add support for display-hint

This commit adds a getter for the display-hint attribute.

GtkModelButton: Add an iconic mode

In iconic mode, model buttons will be styled like regular icon
buttons, preferring to show only the icon if one is set, falling
back to showing the label.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkMenuTrackerItem: Add support for verb-icons

When rendering iconic sections, we want to use icons for verbs,
and we want to differentiate these in the menu model, to keep
the icon attribute reserved for nouns.

https://bugzilla.gnome.org/show_bug.cgi?id=727477

GtkPopover: new approach to menu model binding

Instead of using GtkMenuTracker to flatten the sections into a single
linear menu, handle the sections ourselves by nesting boxes.

Each section gets an inner and outer box.  The inner box numbers its
children in the way that the tracker instructs.  The outer box
containes the inner box and the separator, if appropriate.

Having the two separate boxes will allow us to change the orientation of
the inner box if we want to pack widgets horizontally within a section.

GtkMenuTracker: add 'merge_sections' flag

Add the possibility of a GtkMenuTracker that performs no section
merging.  Instead, it will report an item in the form of a separator for
subsections.  It is then possible to get a separate tracker for the
subsection contents by using gtk_menu_tracker_new_for_item_link().

GtkMenuTracker: don't specialise "submenu" link

We have some API in GtkMenuTracker and GtkMenuTrackerItem that is
specifically designed to deal with submenus.

Generalise these APIs to take a 'link_name' parameter that we always
give as G_MENU_SUBMENU for now.  In the future, this will allow creating
trackers for other types of links, such as sections.

GtkMenuTracker: make "is-visible" a property

Make this a property just like all of the other things and make the APIs
for accessing it non-private (but add a note that they are not intended
to be used).

GtkMenuTracker: remove "visible" property

This is in conflict with the "is-visible" pseudo-property which is about
to be promoted to being an actual property.

W32: Update the css to match the .modelbutton->.button.menuitem transition

Rename the modelbutton style class

This is not a great name to use in themes; instead, add the
menuitem class, so themes can use .button.menuitem for styles
that are specific to menuitem-like buttons.

Add a popover menu example to gtk3-widget-factory

Add a paned example to gtk3-widget-factory

Fix testmenubutton

The menu model examples was messing up the order of its items.

GtkModelButton: improve size allocation

Only reserve size for an indicator if at least one sibling
actually has an indicator.

Actually set the default for GtkMenuButton::use-popover

The previous commit changed the property declaration, but
omitted the actual change of the default.

gdkscreen-wayland: Emit monitors-changed when the output scale changes

https://bugzilla.gnome.org/show_bug.cgi?id=729013

Add Adwaita to the list of default icon themes

Hardcode the fallback to Adwaita. I'm leaving gnome in here for
now, since we're in the middle of the transition. It will be
removed before 3.14.

Fixed crash in GtkLockButton if a button with permission set to NULL is clicked.

https://bugzilla.gnome.org/show_bug.cgi?id=728992

GtkWindow: Avoid a critical in F10 handling

The focus widget might be NULL, and we were forgetting to take
that possibility into account in one place.

tests: add missing files to EXTRA_DIST

https://bugzilla.gnome.org/show_bug.cgi?id=728846

Add missing templates to testdialog

This makes the template test cases not crash. The templates also
serve to test the new action widget support.

https://bugzilla.gnome.org/show_bug.cgi?id=728846

GtkDialog: Update suggested-action more carefully

We were not really handling all cases correctly here. We want
the suggested-action style class to only be set on headerbar
buttons, and it should be set on the default widget. Ensure
this by syncing the suggested-action style class with the
default style class. As a side-effect, setting has-default
on an action widget in ui files will now have the expected
effect.

https://bugzilla.gnome.org/show_bug.cgi?id=728846

GtkDialog: Support adding action widgets in ui files

GtkDialog has convenience API for adding action widgets that are
either placed in the action area or the headerbar. This commit
makes the same functionality available from GtkBuilder ui files
by specifying "action" as the child type.

https://bugzilla.gnome.org/show_bug.cgi?id=728846

openbsd: properly set len for gtkmountoperation-x11

sysctl(3) is the correct way to get the lenght for len in this case.
Also drop unused headers and change style to match the rest of the file.

GtkMenuButton: use popovers by default

When constructing from a menu model, use popovers by default.
This change has the potential to cause some size problems for
applications with big gear menus, so we're doing it early in
the cycle to uncover and fix those.